Add prognostic cumulus closure description in saSAS #1187
Conversation
|
Automated RT Failure Notification |
|
@DeniseWorthen @MinsukJi-NOAA @junwang-noaa @grantfirl RT passes ok! CI is running. Can you leave review comments? |
on-behalf-of @ufs-community <[email protected]>
|
@jkbk2004 @junwang-noaa I'm at a conference today, but I can start the merge process when ready if before 12 ET. After 12 ET, I'll be traveling home for the remainder of the afternoon. If necessary, I can also delegate this to someone else. |
|
@grantfirl can we start merging in ccpp#903 and then fv3atm#523 ? |
I've merged NCAR/ccpp-physics#903 and instructed @lisa-bengtsson to make the changes to her fv3atm branch in anticipation of the merge. |
|
@lisa-bengtsson can you revert the fv3atm pointer here as well? so we can merge in. |
|
@jkbk2004 I updated the .gitmodules and FV3 |
|
Thank you all for all the work on running RT's and merging this! |
…fs-community#1187" https://github.com/ufs-community/ufs-weather-model/pull/1187/files modified: FV3 modified: tests/default_vars.sh modified: tests/opnReqTests/dcp.sh modified: tests/parm/control.nml.IN new file: tests/parm/field_table/field_table_gfsv16_progsigma
…fs-community#1187" (#1) https://github.com/ufs-community/ufs-weather-model/pull/1187/files modified: FV3 modified: tests/default_vars.sh modified: tests/opnReqTests/dcp.sh modified: tests/parm/control.nml.IN new file: tests/parm/field_table/field_table_gfsv16_progsigma
* Adopt progsigma "Add prognostic cumulus closure description in saSAS ufs-community#1187" https://github.com/ufs-community/ufs-weather-model/pull/1187/files modified: FV3 modified: tests/default_vars.sh modified: tests/opnReqTests/dcp.sh modified: tests/parm/control.nml.IN new file: tests/parm/field_table/field_table_gfsv16_progsigma * modified: .gitmodules
* Adopt progsigma "Add prognostic cumulus closure description in saSAS ufs-community#1187" https://github.com/ufs-community/ufs-weather-model/pull/1187/files modified: FV3 modified: tests/default_vars.sh modified: tests/opnReqTests/dcp.sh modified: tests/parm/control.nml.IN new file: tests/parm/field_table/field_table_gfsv16_progsigma * modified: .gitmodules modified: .gitmodules
* Adopt progsigma "Add prognostic cumulus closure description in saSAS ufs-community#1187" https://github.com/ufs-community/ufs-weather-model/pull/1187/files modified: FV3 modified: tests/default_vars.sh modified: tests/opnReqTests/dcp.sh modified: tests/parm/control.nml.IN new file: tests/parm/field_table/field_table_gfsv16_progsigma * modified: .gitmodules modified: .gitmodules
PR Checklist
This PR is up-to-date with the top of all sub-component repositories except for those sub-components which are the subject of this PR. Please consult the ufs-weather-model wiki if you are unsure how to do this.
This PR has been tested using a branch which is up-to-date with the top of all sub-component repositories except for those sub-components which are the subject of this PR
An Issue describing the work contained in this PR has been created either in the subcomponent(s) or in the ufs-weather-model. The Issue should be created in the repository that is most relevant to the changes in contained in the PR. The Issue and the dependent sub-component PR
are specified below.
Results for one or more of the regression tests change and the reasons for the changes are understood and explained below.
New or updated input data is required by this PR. If checked, please work with the code managers to update input data sets on all platforms.
Instructions: All subsequent sections of text should be filled in as appropriate.
The information provided below allows the code managers to understand the changes relevant to this PR, whether those changes are in the ufs-weather-model repository or in a subcomponent repository. Ufs-weather-model code managers will use the information provided to add any applicable labels, assign reviewers and place it in the Commit Queue. Once the PR is in the Commit Queue, it is the PR owner's responsiblity to keep the PR up-to-date with the develop branch of ufs-weather-model.
Description
This PR introduces an optional closure assumption to the Arakawa-Schubert cloud work function in saSAS cumulus convection, in where the mass-flux at cloud base is described as the product between updraft area fraction and updraft velocity. The updraft area fraction is given by a prognostic moisture budget equation.
The update is not aimed for UFS coupled prototype 8, but is an option to be explored beyond P8.
In addition to updates in the ccpp/physics submodule, new field_table, and namelist parameters are added in ufs-weather-model.
The PR is also making a small change in the stochastic_physics_driver.F90 to allow the cellular automata to work on non-uniform blocksizes. Previously the code aborted if not all blocks in the domain had the same size, however, the code works the way it's been formulated, so I simply removed the call to mpp_error in case on non-uniform blocksizes.
##Testing
This PR does not change the answers in the control, but a new test has been added where the scheme is activated using progsigma = True.
The regression tests have been tested on Hera.
Dependencies
Issue: NCAR/ccpp-physics#890
#1193